-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature/#080 panda css 라이브러리 설정 #91
The head ref may contain hidden characters: "Feature/#080_panda_css_\uB77C\uC774\uBE0C\uB7EC\uB9AC_\uC124\uC815"
Conversation
…boostcampwm-2024/web33-Nocta into Feature/#080_panda_css_라이브러리_설정
…into Feature/#080_panda_css_라이브러리_설정 # Conflicts: # pnpm-lock.yaml
- 토큰, 레시피의 경우 다른 파일에서 관리하는게 편하기에 rule 제거 - 단축 속성 사용못하도록 rule 추가 #80
- 색상, 둥글기, 그림자, 간격 토큰 추가 #80
- icon의 경우, 특정 사이즈 (24px, 30px)만을 사용하기에 레시피로 분리해 재활용 #80
- 오히려 구현복잡도가 증가해져서 제거 #80
고생하셨습니다! 네이밍도 깔끔하게 구조별로 잘 지어진것 같습니다! 하나 질문이 있는데 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍👍👍👍👍
conflict 발생했네요 eslint.config.js 부분인듯합니다. 제가 한번 테스트 해볼께요 |
settings: {
"import/resolver": {
typescript: {
alwaysTryTypes: true,
project: resolve(__dirname, "./tsconfig.json"),
},
},
}, 하고 settings: {
"import/resolver": {
typescript: {
alwaysTryTypes: true,
project: "./tsconfig.json",
},
node: {
extensions: [".js", ".jsx", ".ts", ".tsx"],
},
},
"import/parsers": {
"@typescript-eslint/parser": [".ts", ".tsx"],
},
"import/internal-regex": "^@/",
}, 두개를 합쳐서 settings: {
"import/resolver": {
typescript: {
alwaysTryTypes: true,
project: resolve(__dirname, "./tsconfig.json"), // 여기 바뀜
},
react: { // react문 여기추가
version: "detect",
},
node: {
extensions: [".js", ".jsx", ".ts", ".tsx"],
},
},
"import/parsers": {
"@typescript-eslint/parser": [".ts", ".tsx"],
},
"import/internal-regex": "^@/",
}, 위처럼 작성해보았습니다. 확인해보니 에러안나고 저장할때 잘되는데 저렇게 conflict resolve 할까요? |
…into Feature/#080_panda_css_라이브러리_설정 # Conflicts: # client/eslint.config.js
container는 하나의 요소를 담도록, wrapper 여러개 요소의 담도록 해주었습니다! Box는 Container, Wrapper도 아닐때, icon이나 text의 스타일이 필요할때 사용해줬습니다. |
감사합니다! 그런데 프로젝트 경로의 경우, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다!
오우 그런 차이가있엇군요! pnpm lint 와 ctrl+s 가 다르단것도 유의해야겠어요 ! |
아 이제는 둘다 같게 수정했습니다!! |
📝 변경 사항
🔍 변경 사항 설명
디자인 시스템 구축
import문 자동 정렬
panda css eslint + order 정렬 플러그인
bg
와 같은 단축속성을 적을 경우 에러를 띄워주도록 했습니다.css 네이밍
폴더구조
🙏 질문 사항
📷 스크린샷 (선택)
✅ 작성자 체크리스트